Skip to content

fix: add fastify integration to sentry of controlplane#2260

Merged
miklosbarabas merged 2 commits intomainfrom
miklos/fix-controlplane-sentry-add-fastify-integ
Oct 3, 2025
Merged

fix: add fastify integration to sentry of controlplane#2260
miklosbarabas merged 2 commits intomainfrom
miklos/fix-controlplane-sentry-add-fastify-integ

Conversation

@miklosbarabas
Copy link
Copy Markdown
Contributor

@miklosbarabas miklosbarabas commented Oct 3, 2025 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Chores
    • Added Fastify integration to the control plane’s Sentry monitoring.
    • Enhances capture of framework-specific errors and request traces for Fastify endpoints.
    • No changes to user workflows or settings; runtime behavior unchanged.
    • Prepares diagnostics for more consistent and actionable observability across environments.

Checklist

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 3, 2025 •
edited
Loading

Walkthrough

Adds the fastifyIntegration() import and includes fastifyIntegration() in the Sentry.init integrations array in controlplane/src/core/sentry.config.ts. No other logic or public APIs were changed.

Changes

Cohort / File(s) Change summary
Sentry integration update
controlplane/src/core/sentry.config.ts		 Add fastifyIntegration() import from @sentry/node and include fastifyIntegration() in the Sentry.init integrations array alongside existing integrations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly states the main change of adding the Fastify integration to Sentry in the controlplane configuration and is concise and directly related to the code changes.
✨ Finishing touches
  • 📝 Generate docstrings
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1e5d60f and 3cd5f16.

📒 Files selected for processing (1)
  • controlplane/src/core/sentry.config.ts (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: build_push_image
  • GitHub Check: build_test
  • GitHub Check: Analyze (go)
🔇 Additional comments (2)
controlplane/src/core/sentry.config.ts (2)

6-6: LGTM: Import added correctly.

The fastifyIntegration import from @sentry/node is correctly added and follows the existing import pattern in the file.

25-25: fastifyIntegration usage is correct as-is: fastifyIntegration() auto-instruments Fastify and doesn’t accept an app parameter. If you need the Fastify error handler, call Sentry.setupFastifyErrorHandler(app) separately.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@miklosbarabas miklosbarabas enabled auto-merge (squash) October 3, 2025 18:29
@miklosbarabas miklosbarabas merged commit 363b1c3 into main Oct 3, 2025
10 checks passed
@miklosbarabas miklosbarabas deleted the miklos/fix-controlplane-sentry-add-fastify-integ branch October 3, 2025 18:39
@coderabbitai coderabbitai Bot mentioned this pull request Oct 9, 2025
Merged
5 tasks
@coderabbitai coderabbitai Bot mentioned this pull request Nov 16, 2025
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JivusAyrus JivusAyrus JivusAyrus approved these changes

@wilsonrivera wilsonrivera Awaiting requested review from wilsonrivera wilsonrivera is a code owner

@StarpTech StarpTech Awaiting requested review from StarpTech StarpTech is a code owner

@thisisnithin thisisnithin Awaiting requested review from thisisnithin thisisnithin is a code owner

Assignees

No one assigned

Labels

controlplane

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@miklosbarabas @JivusAyrus